-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TestClient.test_service_timestamps failing consistently. #1364
Conversation
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tested this patch with humble with,
export RMW_IMPLEMENTATION=rmw_connextdds
colcon test --event-handlers console_direct+ --retest-until-fail 100 --packages-select rclpy --ctest-args -R test_client
and failure never happened.
Pulls: #1364 |
@fujitatomoya should we backport this to other distros? |
@ahcorde yeah definitely, original issue is for humble. i will manage the backports. thanks! |
@Mergifyio backport humble iron jazzy |
✅ Backports have been created
|
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> (cherry picked from commit 7f9a307)
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> (cherry picked from commit 7f9a307)
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> (cherry picked from commit 7f9a307)
closes #1347
note: backport all distros.